Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move CI to Github Action #94

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Ao-senXiong
Copy link
Member

Other PR fails because of /bin/sh: 1: bazelisk: not found

Use github action will help us consistently handle this.

@wmdietl
Copy link
Member

wmdietl commented Oct 6, 2024

Alternatively, you could make a PR against eisop/checker-framework that adds the package for bazelisk to the Docker images. I'll then re-create the images and this will work.

Going to GitHub Actions might be a got thing anyways, but this PR is currently failing.

@wmdietl
Copy link
Member

wmdietl commented Nov 13, 2024

Can you merge in master to see whether CI is happy now.

@Ao-senXiong
Copy link
Member Author

Updated the code. Interestingly the CI did not start immediately. Maybe the queue of CI runner is organizational level and CF have PRs are running before this CI.

@Ao-senXiong
Copy link
Member Author

Interesting... There must be some error in the CI file then.

cpovirk added a commit to cpovirk/jdk that referenced this pull request Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants